Fix recursive crash prints on FVP AEM model
authorSoby Mathew <[email protected]>
Mon, 13 Apr 2015 15:57:12 +0000 (16:57 +0100)
committerSoby Mathew <[email protected]>
Mon, 13 Apr 2015 16:21:26 +0000 (17:21 +0100)
This patch fixes an issue in the cpu specific register reporting
of FVP AEM model whereby crash reporting itself triggers an exception
thus resulting in recursive crash prints. The input to the
'size_controlled_print' in the crash reporting framework should
be a NULL terminated string. As there were no cpu specific register
to be reported on FVP AEM model, the issue was caused by passing 0
instead of NULL terminated string to the above mentioned function.

Change-Id: I664427b22b89977b389175dfde84c815f02c705a

lib/cpus/aarch64/aem_generic.S

index ee53058ef1a1d5128223d03fe7407d44d3ab6bf0..0ab5253e7f6966bdfd3f042bd5aa42fb0b5f5f07 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014, ARM Limited and Contributors. All rights reserved.
+ * Copyright (c) 2014-2015, ARM Limited and Contributors. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
@@ -79,8 +79,12 @@ endfunc aem_generic_cluster_pwr_dwn
         * reported.
         * ---------------------------------------------
         */
+.section .rodata.aem_generic_regs, "aS"
+aem_generic_regs:  /* The ascii list of register names to be reported */
+       .asciz  "" /* no registers to report */
+
 func aem_generic_cpu_reg_dump
-       mov     x6, #0 /* no registers to report */
+       adr     x6, aem_generic_regs
        ret
 endfunc aem_generic_cpu_reg_dump